-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add node / nvm build error note to the README.md #93
Conversation
I've put it here for now, with the build instructions. If you want more details it might be better on its own page, like a troubleshooting or on the contributing page. To keep the front page README.md from getting too over whelming. |
Yes sounds good. We can just link to the contributing under "compile yourself" on the readme and then put the instructions there. Would be great if you could also figure out the minimal version of Node required for SEA and add it there as well :) |
Thats a good idea, then we can include more details, like we could list minimum versions for all the dependencies if we so wished. I'll do that now. |
Hey, just as a heads-up: I just added Markdown linting to the repo, please rebase to main and make sure your changes will pass the linting rules :) |
72ea75c
to
8d17ebd
Compare
Hi guys, Getting back to this today... Is jq no longer a requirement at all? EDIT: Also is zip needed? or have you switched to tar.gz? (I see zip still referenced in the makefile) |
Awesome! Yes,
|
Cool. Question for both of you @jannis-baum @tuurep, when you are doing ./configure <install_path> ...Where are you guys installing to? I'm installing to ~/.local/bin/ because I have write access there (without needing root) and it is on $path by default, but that might be a Fedora thing, doesn't seem to be the case on Ubuntu. |
We had a discussion about that topic starting in #68 (comment) and the next few comments. tldr; I think Tuure installs to |
Hey this is super out of context but just as a heads-up for both of you @tuurep @Tweekism: I edited your PR descriptions to explicitly say Maybe this is something we should also mention in the contribution guide? @Tweekism could you add it on your PR here since you are already editing that file anyways? |
Sure can mate, I was wondering why I wasn't getting the little icon on the issues page, I thought I had added it. |
Hey peeps, This is ready for you to have a look at, sorry it took so long, since it is a lot of change I know I'm holding up anyone else who wants to update the docs. Is it too much? Note there is one "TODO:" in there, and that is the command to install the required dependencies on MacOS. Could you let me know? Or do we not want to go to that level of detail. I kind of modeled it after the Ladybird browser project that I flirted with a little bit 😅 |
Yes I install to
Same is
Oh.. Interesting lol |
Hey @tuurep, Yeah I went back and had a look at that thread. Is that a default path on Arch? edit: oh you ninja edited me, sneaky sneaky :) |
To be fair, I only checked on Ubuntu server, I’ll have a look on Ubuntu desktop see if it’s any different. |
Actually I went to double check if I have this old line on my
The comment reveals that I didn't know the full extent of what I was doing, but had set up Oh yeah it says user scripts too. Well, it's accurate enough. |
That's interesting haha. I'm in Windows at the moment (downloading Warzone updates 🙄). When I'm back in Linux I'll have a look at my .rc file. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hello! Thanks a lot for putting in the effort of writing such nice instructions here! I appreciate it a lot. See below for comments
Ok I've tightened it up significantly, and re-arranged a bit. I'm going over it again double checking, let me know if thats better or if I missed anything. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice, looks good! Let me know if everything is done from your side as well, if so I will merge this.
I just marked 2 backticks that I missed, other than that I'm happy. |
Oh, did you fix that? I don't see any new changes. |
I marked it in the code review thing but didn't commit it. Maybe you can't see them. Want me to just push a new commit? |
I don't know how to do things on my phone. I have my laptop now give me 2 minutes and I'll fix up these to tiny things |
Done. I can squash those two new tiny commits into the previous one if you like. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
I can squash those two new tiny commits into the previous one if you like.
Yes sounds good!
Hows that look? |
Perfect! Then I will merge it, okay?👀 |
Nice! Thanks a lot again and congrats on your first official contributions to Vivify🥳 |
I'd like to see this project go far because you guys are so nice. I've seen so many small projects die because they were run by one dude who didn't really like to talk to people 😅 |
Thanks! I am also having lots of fun here working with you guys |
Oh, and, speaking of plans, I was thinking to set up a GitHub project board soon where the issues from this and the vivify.vim repo are added automatically and then we can order stuff by what we each want to work on next to keep things more organized. Just as a heads up, you will get an e-mail invitation or something once that's ready :) |
Close #91